css: Remove generic 'none' handling

Includes updated tests.
Who could have thought that our tests were broken. Ooops.

diff --git a/gtk/gtkstyleproperty.c b/gtk/gtkstyleproperty.c
index 464875b7cda3c5a10e054128527face341f8d77a..d139b9da85aab2b88d9205dce3fef296076434ec 100644 (file)
--- a/gtk/gtkstyleproperty.c
+++ b/gtk/gtkstyleproperty.c
@@ -2366,14 +2366,6 @@ _gtk_style_property_parse_value (const GtkStyleProperty *property,
           g_value_set_enum (value, GTK_CSS_INHERIT);
           return TRUE;
         }
-      else if (_gtk_css_parser_try (parser, "none", TRUE))
-        {
-          /* Insert the default value, so it has an opportunity
-           * to override other style providers when merged
-           */
-          g_param_value_set_default (property->pspec, value);
-          return TRUE;
-        }
       else if (property->property_parse_func)
         {
           GError *error = NULL;

diff --git a/tests/css/parser/border-radius.css b/tests/css/parser/border-radius.css
index 72e26b35fc2d36609b0830e14d5329bf3cfdd5ea..7a3312fcf7b47a1d9d43e67449fc9e77029dff47 100644 (file)
--- a/tests/css/parser/border-radius.css
+++ b/tests/css/parser/border-radius.css
@@ -31,34 +31,30 @@ h {
 }
 
 i {
-  border-radius: none;
-}
-
-j {
   border-radius: 1.125 / 5.5;
 }
 
-k {
+j {
   border-radius: 1 2.25 / 5;
 }
 
-l {
+k {
   border-radius: 1 2 3.5 / 5.5 6.75;
 }
 
-m {
+l {
   border-radius: 0 / 0;
 }
 
-n {
+m {
   border-radius: 0;
 }
 
-o {
+n {
   border-radius: 0 / 1;
 }
 
-p {
+o {
   border-radius: 1 / 0;
 }
 

diff --git a/tests/css/parser/border-radius.errors b/tests/css/parser/border-radius.errors
index e6e1134be8663cd84a97d62a061444704bfa05f7..21deded7006671e49a3d75e58b660eb3c9deadcf 100644 (file)
--- a/tests/css/parser/border-radius.errors
+++ b/tests/css/parser/border-radius.errors
@@ -1,3 +1,4 @@
+border-radius.css:64: error: GTK_CSS_PROVIDER_ERROR_SYNTAX
 border-radius.css:68: error: GTK_CSS_PROVIDER_ERROR_SYNTAX
 border-radius.css:72: error: GTK_CSS_PROVIDER_ERROR_SYNTAX
 border-radius.css:76: error: GTK_CSS_PROVIDER_ERROR_SYNTAX
@@ -10,4 +11,3 @@ border-radius.css:100: error: GTK_CSS_PROVIDER_ERROR_SYNTAX
 border-radius.css:104: error: GTK_CSS_PROVIDER_ERROR_SYNTAX
 border-radius.css:108: error: GTK_CSS_PROVIDER_ERROR_SYNTAX
 border-radius.css:112: error: GTK_CSS_PROVIDER_ERROR_SYNTAX
-border-radius.css:116: error: GTK_CSS_PROVIDER_ERROR_SYNTAX

diff --git a/tests/css/parser/border-radius.ref.css b/tests/css/parser/border-radius.ref.css
index a4567e3ce28d8df247f5af4cc0907e0d3f74179c..14f7f0408e07bc8c7c320d7300c1306dff052d22 100644 (file)
--- a/tests/css/parser/border-radius.ref.css
+++ b/tests/css/parser/border-radius.ref.css
@@ -55,55 +55,48 @@ h {
 }
 
 i {
-  border-bottom-left-radius: none;
-  border-bottom-right-radius: none;
-  border-top-left-radius: none;
-  border-top-right-radius: none;
-}
-
-j {
   border-bottom-left-radius: 1.125 5.5;
   border-bottom-right-radius: 1.125 5.5;
   border-top-left-radius: 1.125 5.5;
   border-top-right-radius: 1.125 5.5;
 }
 
-k {
+j {
   border-bottom-left-radius: 2.25 5;
   border-bottom-right-radius: 1 5;
   border-top-left-radius: 1 5;
   border-top-right-radius: 2.25 5;
 }
 
-l {
+k {
   border-bottom-left-radius: 2 6.75;
   border-bottom-right-radius: 3.5 5.5;
   border-top-left-radius: 1 5.5;
   border-top-right-radius: 2 6.75;
 }
 
-m {
+l {
   border-bottom-left-radius: 0;
   border-bottom-right-radius: 0;
   border-top-left-radius: 0;
   border-top-right-radius: 0;
 }
 
-n {
+m {
   border-bottom-left-radius: 0;
   border-bottom-right-radius: 0;
   border-top-left-radius: 0;
   border-top-right-radius: 0;
 }
 
-o {
+n {
   border-bottom-left-radius: 0 1;
   border-bottom-right-radius: 0 1;
   border-top-left-radius: 0 1;
   border-top-right-radius: 0 1;
 }
 
-p {
+o {
   border-bottom-left-radius: 1 0;
   border-bottom-right-radius: 1 0;
   border-top-left-radius: 1 0;

diff --git a/tests/css/parser/border.css b/tests/css/parser/border.css
index 1a0e25248bffbe42145faf1f9e1db2ccd77938d0..3955f688768038490285d3b6bb30fcf199beb65f 100644 (file)
--- a/tests/css/parser/border.css
+++ b/tests/css/parser/border.css
@@ -53,7 +53,3 @@ m {
 n {
   border-property: 1 2 ; 4;
 }
-
-o {
-  border-property: none
-}

diff --git a/tests/css/parser/border.ref.css b/tests/css/parser/border.ref.css
index 7cf5007848dabb38fe65b231b5e6cb5b2aa7d2ec..73bfc35f2da907f646c849191efbcb374b6e5fdd 100644 (file)
--- a/tests/css/parser/border.ref.css
+++ b/tests/css/parser/border.ref.css
@@ -25,7 +25,3 @@ f {
 n {
   border-property: 1 2;
 }
-
-o {
-  border-property: none;
-}

diff --git a/tests/css/parser/font-family.ref.css b/tests/css/parser/font-family.ref.css
index b770bf489dd88a1e01c82e2d61bb601252760b7a..319f3d074d9244dc92ccd369f9c6bfaf47a6229b 100644 (file)
--- a/tests/css/parser/font-family.ref.css
+++ b/tests/css/parser/font-family.ref.css
@@ -1,5 +1,5 @@
 a {
-  font-family: none;
+  font-family: "none";
 }
 
 b {